Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

fix: fixed issue 56 - error to encode/decode private key. #57

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

joaoluizcadore
Copy link

Fixed the issue 56. The problem was related to save the key in the config.json file. It was decoding the string before save in the config.json file, but when it reads the key from the file, it tries to decode the key again. Not sure what is correct, but it fixed the issue. In my tests, now it is possible to use the application.

@fiatjaf
Copy link
Owner

fiatjaf commented May 2, 2023

I think the error is elsewhere. Can't be this. Maybe there is an error when writing the key to the config file. It should encode to hex first.

@joaoluizcadore
Copy link
Author

The pull request before mine fixed this issue. I used his solution to fix this in a better way.
Also added a json tag called "content_decrypted" when an event of KindEncryptedDirectMessage type is printed using json format (--json flag).

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants